CSPL-4358 Split BusConfiguration CR into 2 CRs - Queue and ObjectStorage#1643
Open
kasiakoziol wants to merge 27 commits intoCSPL-3551-ingestion-crfrom
Open
CSPL-4358 Split BusConfiguration CR into 2 CRs - Queue and ObjectStorage#1643kasiakoziol wants to merge 27 commits intoCSPL-3551-ingestion-crfrom
kasiakoziol wants to merge 27 commits intoCSPL-3551-ingestion-crfrom
Conversation
Collaborator
Pull Request Test Coverage Report for Build 21821066449Details
💛 - Coveralls |
…PL-4360-secret-ref-ingest-index
kasiakoziol
changed the title
5 tasks
kasiakoziol
requested review from
Igor-splunk,
kubabuczak,
patrykw-splunk,
rlieberman-splunk and
vivekr-splunk
rlieberman-splunk
approved these changes
Jan 8, 2026
| // +optional | ||
| // +kubebuilder:validation:Pattern=`^https?://[^\s/$.?#].[^\s]*$` | ||
| // S3-compatible Service endpoint | ||
| Endpoint string `json:"endpoint"` |
Collaborator
There was a problem hiding this comment.
are we expecting the endpoint to have the region. do we need region separately.
Collaborator
There was a problem hiding this comment.
ObjectStorage doesn't have region field, relies on Path. But if Endpoint is constructed automatically somewhere, same China/GovCloud issue from PR
Collaborator
Author
There was a problem hiding this comment.
The region in ObjectStorage is taken from Queue if it is provided over there
vivekr-splunk
approved these changes
Feb 6, 2026
Collaborator
vivekr-splunk
left a comment
vivekr-splunk
left a comment
There was a problem hiding this comment.
i have approve with comments
vivekr-splunk
approved these changes
Feb 6, 2026
Collaborator
vivekr-splunk
left a comment
vivekr-splunk
left a comment
There was a problem hiding this comment.
approved with comments
Collaborator
Author
|
[EDIT: Addressed] |
…est-index Feature/CSPL-4360 Secret reference for Index & Ingestion separation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends On
Description
It splits BusConfiguration CR into 2 CRs - Queue and ObjectStorage as per internal documentation.
Key Changes
Testing and Verification
Related Issues
Jira: https://splunk.atlassian.net/browse/CSPL-4358
PR Checklist